Skip to content
This repository was archived by the owner on May 29, 2026. It is now read-only.

Potential fix for code scanning alert no. 31: Disallow the any type#33

Merged
Jackson57279 merged 1 commit into
mainfrom
alert-autofix-31
Jul 20, 2025
Merged

Potential fix for code scanning alert no. 31: Disallow the any type#33
Jackson57279 merged 1 commit into
mainfrom
alert-autofix-31

Conversation

@Jackson57279

Copy link
Copy Markdown
Collaborator

Potential fix for https://github.com/otdoges/zapdev/security/code-scanning/31

To fix the problem, we should replace the any type in the callback argument with Supabase’s official session type. Supabase’s JS client exports a Session type that describes the shape of the session object. We need to import Session from @supabase/supabase-js and update the callback signature to use (event: string, session: Session | null) => void, since the session can be null when the user is not signed in. The edit is required in src/lib/supabase.ts at line 64. We will need to add the import for Session at the top of the file.

Suggested fixes powered by Copilot Autofix. Review carefully before merging.

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@vercel

vercel Bot commented Jul 20, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
zapdev ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 20, 2025 3:02pm

@Jackson57279 Jackson57279 marked this pull request as ready for review July 20, 2025 15:02
@Jackson57279 Jackson57279 merged commit 38b650f into main Jul 20, 2025
9 checks passed
@Jackson57279 Jackson57279 deleted the alert-autofix-31 branch July 20, 2025 15:03
Jackson57279 pushed a commit that referenced this pull request Aug 30, 2025
Add support for Google Gemini AI
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant